Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEAT] 공홈 어드민 데이터 연결 #435

Merged
merged 16 commits into from
Jan 1, 2025
Merged

Conversation

lydiacho
Copy link
Member

@lydiacho lydiacho commented Dec 29, 2024

Summary

  • 이미지로 박던 커리큘럼쪽 컴포넌트 퍼블리싱
  • 어드민 배포 못하는 동안 mock 데이터 만들어서 쓰다가 현재는 remoteAPI로 다시 적용해둠
  • 공홈 어드민 데이터 api 연결
  • 어드민으로 dev 서버에 데이터 배포 후, 잘 받아오는거 확인 + 반영

Screenshot

스크린샷 2025-01-01 오전 4 26 37

Comment

몇가지 참고사항이 있어요

  • RecruitPage에 기존에 lazy loading을 적용해주던 하단 컴포넌트들의 lazy loading을 잠시 주석처리해두었어요. 서버 데이터랑 연결하려니 오류가 나더라구요. 어떻게 해결해야 할지 빠르게 떠오르진 않아서 일단 주석처리 해뒀고 추후 다시 최적화하면 좋을 것 같습니다.
  • ssg(isr)로 불러오던 about 탭에 작업을 해주니까 prerender 에러가 발생해서 빌드가 깨지더라구요.. 그래서 우선 ssg 빼주었어요 추후 다시 고민해보려고 합니다
  • 소개탭 > 이전 기수 레코드 정보는 서버측에 같이 보내달라고 요청할 예정이라 잠시 주석처리해두었어요
  • 현재 이미지는 뜨지 않아요. 서버쪽 로직이 잘못된 것 같아서 수정 요청 드린 상태입니다.
  • TODO 주석 몇개 달아놓았어요. 그중 가장 빠르게 고쳐져야 할 건 현재 지원페이지 getApi에서만 모집일정을 전달해주고, homepage APi에서는 전달 안해주는데, 홈 화면에서도 모집 중인지 여부를 서버 데이터 기반으로 체킹해야 해서 이것도 API 확장 요청 드려놓았습니다. API 확장되면 이후에 반영할게요!

Copy link

height bot commented Dec 29, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@lydiacho lydiacho marked this pull request as ready for review December 31, 2024 19:25
@lydiacho lydiacho changed the title Feat/#434 admin api [FEAT] 공홈 어드민 데이터 연결 Dec 31, 2024
@lydiacho lydiacho merged commit da5f9f9 into develop Jan 1, 2025
1 check passed
@lydiacho lydiacho deleted the feat/#434_admin-api branch January 1, 2025 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant